Fix default embedding model in README and enforce LF line endings#37
Merged
carlos-alm merged 2 commits intomainfrom Feb 23, 2026
Merged
Fix default embedding model in README and enforce LF line endings#37carlos-alm merged 2 commits intomainfrom
carlos-alm merged 2 commits intomainfrom
Conversation
The release step was staging all files, which committed native .node binaries and pkg/ directories into the release branch. Now only package.json, package-lock.json, CHANGELOG.md, and DEPENDENCIES.md are staged for the version bump commit.
…dings The README models table incorrectly marked minilm as the default embedding model. The actual default is jina-code (config.js, embedder.js, cli.js). Also adds .gitattributes to enforce LF line endings, fixing 32 Biome format errors caused by CRLF on Windows checkouts.
Contributor
Greptile SummaryThis PR fixes a documentation error in the README and adds line ending enforcement to prevent formatting issues. The README incorrectly labeled Key changes:
Confidence Score: 5/5
Important Files Changed
Last reviewed commit: c852707 |
3 tasks
carlos-alm
added a commit
that referenced
this pull request
Mar 21, 2026
- GitNexus overall score corrected from 4.7 to 4.5 to match the arithmetic mean of its six sub-scores (5+5+4+4+4+5)/6 = 4.5 - Tier 2 renumbered starting at #38 (was duplicating #37 with Tier 1); also resolves the pre-existing duplicate #43 (Bikach/ChrisRoyse now #44/#45), with all subsequent entries incremented accordingly - jelly section header updated from 417 to 423 stars to match the ranking table
carlos-alm
added a commit
that referenced
this pull request
Mar 23, 2026
…#559) * docs: update competitive analysis for v3.2.0 and March 2026 landscape Re-rank codegraph from #8 (4.0) to #5 (4.5) reflecting v3.2.0 features: 41 CLI commands, 32 MCP tools, dataflow across all 11 languages, CFG, sequence diagrams, architecture boundaries, unified graph model. Add new competitors: GitNexus (#1, 18k stars), DeusData/codebase-memory-mcp (#6, 793 stars in 25 days). Update star counts and feature status across all 85+ ranked projects. Mark 7 roadmap items as DONE. Flag stagnant projects. Update joern.md (3,021 stars, 75 contributors, 4 community MCP wrappers) and narsil-mcp.md (129 stars, SPA frontend, +36 security rules, development paused since Feb 25). * docs: fix narsil SPA version attribution in competitive analysis overview Line 18 incorrectly stated "v1.6.1" as the version when the SPA feature was introduced. The SPA frontend was added in v1.6.0; v1.6.1 is the current release. Updated to "added v1.6.0, current v1.6.1" to match the detailed narsil-mcp.md entry. * docs: remove hardcoded star count from joern comparison table The "32 stars, growing" value in the Community & maturity row hardcodes a stale star count. Other comparison tables use "Growing" consistently for codegraph's community status. Updated to match. * fix: correct GitNexus score, Tier 2 rank numbering, and jelly star count - GitNexus overall score corrected from 4.7 to 4.5 to match the arithmetic mean of its six sub-scores (5+5+4+4+4+5)/6 = 4.5 - Tier 2 renumbered starting at #38 (was duplicating #37 with Tier 1); also resolves the pre-existing duplicate #43 (Bikach/ChrisRoyse now #44/#45), with all subsequent entries incremented accordingly - jelly section header updated from 417 to 423 stars to match the ranking table * fix: correct aider rank and codegraph star count per review feedback * fix: align scoring breakdown sub-scores with overall rankings for stagnant projects glimpse: Community 4→2 (stagnant since Jan 2026), avg now 3.83≈3.8 matching ranking. autodev-codebase: Community 3→1 (stagnant since Jan 2026), avg now 3.33, ranking updated 3.4→3.3. * fix: align ranking scores with sub-score averages for colbymchenry and axon * fix: correct ranking inversion at positions #23/#24 (#559) autodev-codebase (3.3) was ranked #23 above Claude-code-memory (3.4) at #24. Swapped to maintain descending score order. * fix: correct score mismatches for code-graph-rag (4.5→4.2) and arbor (3.7→4.2) (#559) * fix: sync breakdown table row order with ranking table for #23/#24 (#559) * fix: correct ranking inversions and stale rank references (#559) * fix: correct sub-score/overall-score mismatches for codexray, loregrep, MATE * fix: correct score mismatches and aider header rank * fix: update narsil-mcp Key Metrics to reflect development stagnation (#559) * fix: add missing "vs arbor" comparison section (#559) * fix: remove duplicate vs-glimpse section and correct role names in vs-arbor (#559) The duplicate vs-glimpse block (stale rank #10) was left behind when vs-arbor was inserted. Removed it — the correct version exists at #11. Also fixed role vocabulary in vs-arbor: bridge → adapter, added entry.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
minilmas the default embedding model — the actual default isjina-code(as set inconfig.js,embedder.js, andcli.js).gitattributeswith* text=auto eol=lfto prevent CRLF line endings on Windows, which were causing 32 Biome format errorsTest plan
npm run lintpasses cleanjina-code (default).gitattributesenforces LF on fresh clone